Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bitnami/kube-state-metrics] Add tests and publishing using VIB #13335

Merged
merged 2 commits into from
Nov 8, 2022

Conversation

joancafom
Copy link
Contributor

Description of the change

The main objective of this PR is to publish our Bitnami Kube State Metrics Helm Chart using VMware Image Builder. In order to do that, several changes are included:

  • Adding the asset to the publishing workflow
  • Increasing the existing test coverage of the asset by adding Goss tests.

Benefits

  • Ensuring higher quality of the Helm charts.
  • Increased pool of assets completely handled by VMware Image Builder.

Possible drawbacks

Automated tests could show to be potentially flaky.

Applicable issues

NA

Additional information

Tested in joancafom#96

Checklist

Signed-off-by: Jose Antonio Carmona <jcarmona@vmware.com>
Signed-off-by: Jose Antonio Carmona <jcarmona@vmware.com>
@bitnami-bot bitnami-bot added the verify Execute verification workflow for these changes label Nov 3, 2022
@joancafom joancafom removed the request for review from CeliaGMqrz November 3, 2022 15:32
Copy link
Contributor

@FraPazGal FraPazGal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@FraPazGal FraPazGal assigned joancafom and unassigned agomezmoron and FraPazGal Nov 7, 2022
@joancafom joancafom merged commit 0066237 into main Nov 8, 2022
@joancafom joancafom deleted the add-kube-state-metrics-tests branch November 8, 2022 09:36
@github-actions github-actions bot added the solved label Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitnami solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants